feat: Implement secure admin key update with two-step process#29
Merged
JerryIdoko merged 2 commits intoVesting-Vault:mainfrom Feb 21, 2026
Merged
Conversation
added 2 commits
February 20, 2026 04:38
- Add propose_new_admin function for creating pending transfer proposals - Add accept_ownership function to complete transfer process - Add transfer_ownership function for backward compatibility - Add comprehensive API endpoints for admin key management - Include 24-hour expiration on pending proposals - Add full audit logging for all admin actions - Support both global and contract-specific admin transfers - Add comprehensive test suite with error handling - Prevent accidental lockout with two-step verification Resolves Vesting-Vault#16: [Admin] Update Admin Key
- Add Vault, SubSchedule, and Beneficiary models - Implement complex cliff logic for multiple top-ups - Add comprehensive vesting service with withdrawal processing - Create REST API endpoints for vault management - Add comprehensive unit and integration tests - Include detailed documentation and API reference Closes Vesting-Vault#19
Contributor
|
Resolve conflicts |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
close #16 Issue 16: [API] Portfolio View Aggregation #16